Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

harfbuzz/10.2.0 package update #39343

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

octo-sts[bot]
Copy link
Contributor

@octo-sts octo-sts bot commented Jan 11, 2025

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr labels Jan 11, 2025
Copy link
Contributor Author

octo-sts bot commented Jan 11, 2025

⚠️ EXPERIMENTAL

Please use 👍 or 👎 on this comment to indicate if you agree or disagree with the recommendation.

To provide more detailed feedback please comment on the recommendation prefixed with /ai-verify:

e.g. /ai-verify partially helpful but I also added bash to the build environment

Gen AI suggestions to solve the build error:

Based on the error output, I'll provide a structured analysis and solution:

• Detected Error: src/meson.build:3:15: ERROR: Command `/home/build/src/gen-hb-version.py 10.2.0 /home/build/output/src/hb-version.h /home/build/src/hb-version.h.in` failed with status 127

• Error Category: Build Configuration

• Failure Point: Python script execution during Meson configuration phase

• Root Cause Analysis: Status code 127 typically indicates "command not found" which suggests Python execution environment issues. The gen-hb-version.py script can't execute properly despite being found by Meson.

• Suggested Fix:
Add Python 3 as a build dependency in the environment section:

environment:
  contents:
    packages:
      - python3
      # ... existing packages ...

• Explanation: The harfbuzz build system requires Python 3 to generate version-specific headers. Status 127 indicates the Python interpreter isn't available in the build environment despite the script being present.

• Additional Notes:

  1. Python is required for the build-time version header generation
  2. The script was found but couldn't execute, indicating missing runtime
  3. No need to modify the script path as Meson found it correctly
  4. Adding Python3 as a build dependency resolves the execution environment

• References:

@octo-sts octo-sts bot added the ai/skip-comment Stop AI from commenting on PR label Jan 11, 2025
the AI suggested comment was right this time, the build was invoking a
python script during build and python was not available, adding python3
builds the package.

Signed-off-by: kranurag7 <[email protected]>
@octo-sts octo-sts bot added bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. manual/review-needed labels Jan 11, 2025
@kranurag7 kranurag7 self-assigned this Jan 11, 2025
@kranurag7 kranurag7 requested a review from a team January 13, 2025 04:57
@philroche philroche merged commit c891698 into main Jan 13, 2025
14 checks passed
@philroche philroche deleted the wolfictl-e7454ebb-ef5b-4cf4-a4d0-11524be70fa9 branch January 13, 2025 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ai/skip-comment Stop AI from commenting on PR automated pr bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. manual/review-needed request-version-update request for a newer version of a package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants